Skip to content

chore: raise native dep versions for minimum recording duration#394

Merged
ioannisj merged 1 commit into
mainfrom
feat/minimum-recording-duration
May 26, 2026
Merged

chore: raise native dep versions for minimum recording duration#394
ioannisj merged 1 commit into
mainfrom
feat/minimum-recording-duration

Conversation

@ioannisj
Copy link
Copy Markdown
Contributor

💡 Motivation and Context

Docs PR: PostHog/posthog.com#17015
Posthog PR: PostHog/posthog#59942

Session replay's minimum recording duration feature is already implemented in native iOS and Android SDKs and works transparently through the Flutter wrapper. Flutter-captured $snapshot events flow through the native replay queue, which is gated by the same buffering logic and delegate that enforces the minimum duration.

No additional Flutter-side code is required for the behavior.

However, the declared native dependency floors were inconsistent:

  • iOS already pins >= 3.56.0, comfortably above the iOS feature floor (3.53.0).
  • Android pins >=3.38.0 which is below the Android feature floor (3.44.0).

💚 How did you test it?

  • Tested that minimum recording duration is respected on both iOS and Android sample apps

📝 Checklist

  • I reviewed the submitted code.
  • I added tests to verify the changes.
  • I updated the docs if needed.
  • No breaking change or entry added to the changelog.

If releasing new changes

  • Ran pnpm changeset to generate a changeset file

@ioannisj ioannisj requested a review from a team as a code owner May 25, 2026 19:06
@ioannisj ioannisj merged commit f9490ed into main May 26, 2026
19 checks passed
@ioannisj ioannisj deleted the feat/minimum-recording-duration branch May 26, 2026 10:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants